Skip to content

fix: Remove gomod private#77

Merged
js-murph merged 2 commits intomainfrom
johnm/remove-gomod-private
Feb 3, 2026
Merged

fix: Remove gomod private#77
js-murph merged 2 commits intomainfrom
johnm/remove-gomod-private

Conversation

@js-murph
Copy link
Copy Markdown
Contributor

@js-murph js-murph commented Feb 3, 2026

What?

Removes the gomod private implementation

Why?

It was deeply flawed and we need to modify our github caching for this to work

@js-murph js-murph requested a review from a team as a code owner February 3, 2026 00:15
@js-murph js-murph requested review from alecthomas and removed request for a team February 3, 2026 00:15
Copy link
Copy Markdown
Contributor

@nssherpa nssherpa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't look deeply as I'm assuming this was just a git revert..

@js-murph js-murph merged commit 7050715 into main Feb 3, 2026
5 checks passed
@js-murph js-murph deleted the johnm/remove-gomod-private branch February 3, 2026 01:33
js-murph added a commit that referenced this pull request Feb 5, 2026
# What?

Re-implements #77, which adds
support for private gomod repositories.

# Why?

So that we are able to cache private go modules.

# Tests?

Includes new unit tests. Additionally some manual testing has been
performed...

```bash
time curl http://localhost:8080/gomod/github.com/<org>/<repo>/@latest
{"Version":"v0.0.6","Time":"2021-08-12T18:29:12Z"}
real	1m43.818s
user	0m0.009s
sys	0m0.017s

time curl http://localhost:8080/gomod/github.com/<org>/<repo>/@latest
{"Version":"v0.0.6","Time":"2021-08-12T18:29:12Z"}
real	0m0.070s
user	0m0.006s
sys	0m0.011s
```

---------

Co-authored-by: Alec Thomas <aat@block.xyz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants